Skip to content

fix gltf path texture usages#22086

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
robtfm:gltf-path-texture-usages
Dec 11, 2025
Merged

fix gltf path texture usages#22086
alice-i-cecile merged 2 commits intobevyengine:mainfrom
robtfm:gltf-path-texture-usages

Conversation

@robtfm
Copy link
Contributor

@robtfm robtfm commented Dec 11, 2025

Objective

the asset usages from GltfLoaderSettings::load_materials are not respected when loading path-based / non-embedded textures.

Solution

use them

@robtfm robtfm added C-Bug An unexpected or incorrect behavior A-glTF Related to the glTF 3D scene/model format labels Dec 11, 2025
@alice-i-cecile alice-i-cecile added this to the 0.18 milestone Dec 11, 2025
@alice-i-cecile alice-i-cecile added the S-Needs-Review Needs reviewer attention (from anyone!) to move forward label Dec 11, 2025
@alice-i-cecile alice-i-cecile added D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Dec 11, 2025
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Dec 11, 2025
Merged via the queue into bevyengine:main with commit e4bf6dc Dec 11, 2025
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-glTF Related to the glTF 3D scene/model format C-Bug An unexpected or incorrect behavior D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants